Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

post-merge updates: consensus mechanisms #7143

Merged
merged 5 commits into from
Aug 1, 2022

Conversation

jmcook1186
Copy link
Contributor

@jmcook1186 jmcook1186 commented Jul 20, 2022

Description

updates consensus-mechanisms page for post merge

Related Issue

#7075

@gatsby-cloud
Copy link

gatsby-cloud bot commented Jul 20, 2022

Gatsby Cloud Build Report

post-merge

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 2m

Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jmcook1186 Left a few notes =)

src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved

## Types of consensus mechanisms {#types-of-consensus-mechanisms}

### Proof-of-work {#proof-of-work}
### Proof-of-work based {#proof-of-work}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️

src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
Copy link
Contributor

@minimalsm minimalsm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice :-) noticed one section that we still talk about proof-of-work as the block creation method, other than that just a few wording suggestions that you can take or leave.

src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
src/content/developers/docs/consensus-mechanisms/index.md Outdated Show resolved Hide resolved
@minimalsm minimalsm merged commit ee597e2 into post-merge Aug 1, 2022
@minimalsm minimalsm deleted the post-merge-consensus-mechanisms branch August 1, 2022 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants